Skip to content

test: dogfood codegraph-pr action#4

Merged
anvanster merged 2 commits into
mainfrom
test/dogfood-pr-action
May 28, 2026
Merged

test: dogfood codegraph-pr action#4
anvanster merged 2 commits into
mainfrom
test/dogfood-pr-action

Conversation

@anvanster
Copy link
Copy Markdown
Member

Summary

Test PR to validate the new codegraph-pr GitHub Action end-to-end.

Touches emit_tel (a function with many callers across the telemetry
choke points) with a docs-only change, so the action's pr_context
output should show:

  • blast radius (emit_tel's callers)
  • change classification (body_changed, docs-only)
  • test gaps (emit_tel has no direct test caller)

Test plan

  • CodeGraph PR Review action runs on ubuntu-latest
  • npm install pulls 0.17.5 (has --graph-only/--run-tool)
  • Graph-only index completes (no embeddings)
  • A markdown review comment is posted

🤖 This PR exists to test the action; safe to close after verifying.

anvanster and others added 2 commits May 28, 2026 10:42
Test PR to dogfood the codegraph-pr GitHub Action.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
The dogfood Action failed: in a PR checkout the base branch exists only
as a remote-tracking ref (origin/main), not a local branch, so
`git diff main...HEAD` errored with "unknown revision".

Two fixes:
1. pr_context now resolves the base ref — tries the bare ref first
   (works locally), falls back to origin/<ref> (works in CI). Users
   can pass plain "main" anywhere.
2. Workflow passes origin/<base> explicitly so it works with the
   already-published 0.17.5 binary (which lacks the auto-resolution),
   and no longer hides stderr — failures now show in the Actions log
   with a ::warning:: annotation instead of a silent "review failed".

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

🔍 CodeGraph PR Review

2 files changed (+51/−6, 2 functions) · Risk: 🟢 low

Blast radius

5 direct callers affected across codegraph-server/src/mcp

⚠️ Test gaps (2 functions, 0 coverage)

  • emit_tel (crates/codegraph-server/src/mcp/server.rs) — signature_changed
  • execute_tool_inner (crates/codegraph-server/src/mcp/server.rs) — body_changed

Suggested reviewers

Andrey Vasilevsky (27 lines)

Suggested commit: feat(mcp): <describe the change> · 0 tests cover the changes
🤖 Generated by CodeGraph

@codegraph-ai codegraph-ai deleted a comment from github-actions Bot May 28, 2026
@anvanster anvanster merged commit 2969833 into main May 28, 2026
1 check passed
@anvanster anvanster deleted the test/dogfood-pr-action branch May 28, 2026 17:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant